symboliccolor: Parse 'currentColor' everywhere
authorBenjamin Otte <otte@redhat.com>
Thu, 5 Apr 2012 20:22:52 +0000 (22:22 +0200)
committerBenjamin Otte <otte@redhat.com>
Tue, 17 Apr 2012 06:59:20 +0000 (08:59 +0200)
gtk/gtkcssshadowvalue.c
gtk/gtkcssshorthandpropertyimpl.c
gtk/gtkcssstylefuncs.c
gtk/gtkcssstylepropertyimpl.c
gtk/gtksymboliccolor.c

index e9cbd4db43dd5192eef99ef5df4022f99f396e4e..8efcbe47417f27585e081a14a00c1ba2da9d2525 100644 (file)
@@ -236,10 +236,7 @@ _gtk_css_shadow_value_parse (GtkCssParser *parser)
       {
         GtkSymbolicColor *symbolic;
 
-        if (_gtk_css_parser_try (parser, "currentcolor", TRUE))
-          symbolic = gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
-        else
-          symbolic = _gtk_css_parser_read_symbolic_color (parser);
+        symbolic = _gtk_css_parser_read_symbolic_color (parser);
         if (symbolic == NULL)
           goto fail;
 
index b86b76110b9056904f395a60a09b106133a3ea23..5ce30967568bf325dd26ca63f4cb51aca88a0ba8 100644 (file)
@@ -221,16 +221,9 @@ parse_border_color (GtkCssShorthandProperty  *shorthand,
 
   for (i = 0; i < 4; i++)
     {
-      if (_gtk_css_parser_try (parser, "currentcolor", TRUE))
-        {
-          symbolic = gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
-        }
-      else
-        {
-          symbolic = _gtk_css_parser_read_symbolic_color (parser);
-          if (symbolic == NULL)
-            return FALSE;
-        }
+      symbolic = _gtk_css_parser_read_symbolic_color (parser);
+      if (symbolic == NULL)
+        return FALSE;
 
       values[i] = _gtk_css_value_new_take_symbolic_color (symbolic);
 
index 3315fbfdbc77d7ca773df68c50144925051c08cf..b1a097e90d75da640c5f07faa8b4ba45498f7f0b 100644 (file)
@@ -169,16 +169,9 @@ rgba_value_parse (GtkCssParser *parser,
   GtkSymbolicColor *symbolic;
   GdkRGBA rgba;
 
-  if (_gtk_css_parser_try (parser, "currentcolor", TRUE))
-    {
-      symbolic = gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
-    }
-  else
-    {
-      symbolic = _gtk_css_parser_read_symbolic_color (parser);
-      if (symbolic == NULL)
-        return FALSE;
-    }
+  symbolic = _gtk_css_parser_read_symbolic_color (parser);
+  if (symbolic == NULL)
+    return FALSE;
 
   if (gtk_symbolic_color_resolve (symbolic, NULL, &rgba))
     {
@@ -312,16 +305,9 @@ symbolic_color_value_parse (GtkCssParser *parser,
 {
   GtkSymbolicColor *symbolic;
 
-  if (_gtk_css_parser_try (parser, "currentcolor", TRUE))
-    {
-      symbolic = gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
-    }
-  else
-    {
-      symbolic = _gtk_css_parser_read_symbolic_color (parser);
-      if (symbolic == NULL)
-        return FALSE;
-    }
+  symbolic = _gtk_css_parser_read_symbolic_color (parser);
+  if (symbolic == NULL)
+    return FALSE;
 
   g_value_take_boxed (value, symbolic);
   return TRUE;
index 2f365ead22dd67cd600e77f9dd22865aec719e2b..58be0bc143e61dabd94a6787646d4a599db6354b 100644 (file)
@@ -183,16 +183,9 @@ color_parse (GtkCssStyleProperty *property,
 {
   GtkSymbolicColor *symbolic;
 
-  if (_gtk_css_parser_try (parser, "currentcolor", TRUE))
-    {
-      symbolic = gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
-    }
-  else
-    {
-      symbolic = _gtk_css_parser_read_symbolic_color (parser);
-      if (symbolic == NULL)
-        return NULL;
-    }
+  symbolic = _gtk_css_parser_read_symbolic_color (parser);
+  if (symbolic == NULL)
+    return NULL;
 
   return _gtk_css_value_new_take_symbolic_color (symbolic);
 }
index 994b2302f0696bab2754228ce84122349fe719e5..a660b8813de3a585d5fd66065ced4f33987b1863 100644 (file)
@@ -1039,6 +1039,9 @@ _gtk_css_parser_read_symbolic_color (GtkCssParser *parser)
 
   g_return_val_if_fail (parser != NULL, NULL);
 
+  if (_gtk_css_parser_try (parser, "currentColor", TRUE))
+    return gtk_symbolic_color_ref (_gtk_symbolic_color_get_current_color ());
+
   if (_gtk_css_parser_try (parser, "transparent", TRUE))
     {
       GdkRGBA transparent = { 0, 0, 0, 0 };